Skip to content

Conversation

@J-Jaywalker
Copy link
Contributor

@J-Jaywalker J-Jaywalker commented Aug 13, 2025

Apologies for being back at it again.

When testing multichannel internally, I noticed that many people instinctively looked for the channel field within the root or metadata of the AddTranscript message. Right now, channel is in both the results and metadata object. This change will mean the Python client will look for it in just the root of the message.

Accompanying change for this can be found in GitLab:
https://gitlab1.speechmatics.io/aml/aladdin/-/merge_requests/10861

Have tested the changes against the branch above, rough local testing with 5 channels transcribing the same audio file.

Without:

It
It
is up
It
It
It
is up
is up
to a new

With new changes:

channel_5: It
channel_5: is up
channel_1: It
channel_5: to a new
channel_4: It
channel_1: is up
channel_4: is up
  channel_2: It
channel_4: to a new
channel_3: It
channel_1: to a new

@J-Jaywalker
Copy link
Contributor Author

Would this need a release tag to get put into Aladdin?

@dumitrugutu
Copy link
Contributor

Would this need a release tag to get put into Aladdin?

Yes, and since it's a potentially a breaking change it should be released with major version bump

Co-authored-by: Dumitru Gutu <dumitrugutu@users.noreply.github.com>
@J-Jaywalker J-Jaywalker merged commit 04cca7a into master Aug 13, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants